Skip to content

Conversation

@HowardBraham
Copy link
Contributor

@HowardBraham HowardBraham commented Jan 8, 2026

Used in MetaMask/metamask-extension#39116

Note

Introduces a configurable merge strategy to the merge-approved-pr.yml workflow.

  • Adds optional merge-method input (default merge; supports merge or squash) with validation in Validate Inputs
  • Updates merge step to pass merge_method: '${{ inputs.merge-method }}' to github.rest.pulls.merge
  • Adjusts logging/messages to include selected merge method

Written by Cursor Bugbot for commit 58ce776. This will update automatically on new commits. Configure here.

repo: context.repo.repo,
pull_number: ${{ inputs.pr-number }},
merge_method: 'merge'
merge_method: '${{ inputs.merge-method }}'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be provided as environment variable (as well as the PR number, though not strictly related to this PR).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh sorry, actually I'm going to close this PR, we're going with a different solution to this issue

@HowardBraham
Copy link
Contributor Author

Going with a different solution to this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants